Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding position-based roaming. #62

Draft
wants to merge 5 commits into
base: iron
Choose a base branch
from

Conversation

fbrand-new
Copy link
Contributor

(AP = Access Point)

Added

Roaming functionalities

  • Using a client/server architecture. The server depends on the navigation and will publish on a yarp port the AP to roam to.
  • The current logic is to roam to the closest available AP, whenever the robot is further away than a parameterized threshold from the currently connected AP.
  • The client reads the AP and roams. The client should run directly on the host machine and not inside the docker. It also need sudo privileges.
  • Added unit tests and corresponding mock classes.
  • The server depends on iwconfig program.
  • The client depends on libwpa_client.so that should be installed on the host machine.

Modified

Dockerfile

  • Added wireless-tools package to the recipe (in order to have iwconfig)

Still missing

  • Documentation regarding required setup on host machine
  • Testing on the real robot. Tests were performed in simulation.

@randaz81 randaz81 marked this pull request as draft June 3, 2024 09:17
- Unit tests
- Roaming functionality independent of navigation

Missing:
- Deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant